From: tsteven4 <13596209+tsteven4@users.noreply.github.com>
Date: Thu, 31 Oct 2024 16:06:10 +0000 (-0600)
Subject: clang tidy fixes (#1365)
X-Git-Tag: archive/raspbian/1.10.0+ds-2+rpi1~1^2~12^2^2~24
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=74cc156003f88c83fd7af6ae61988855ea8f852a;p=gpsbabel.git

clang tidy fixes (#1365)

* fix tidy warings related to passing const container.

readability-avoid-const-params-in-decls
performance-unnecessary-value-param

* tidy readability-inconsistent-declaration-parameter-name

* tidy cppcoreguidelines-prefer-member-initializer

* tidy readability-redundant-member-init

* tidy modernize-loop-convert

* tidy modernize-type-traits

* tidy readability-named-parameter
---

diff --git a/bend.h b/bend.h
index fd7b101d5..4e2cfda11 100644
--- a/bend.h
+++ b/bend.h
@@ -64,7 +64,7 @@ private:
     },
   };
 
-  Waypoint* create_wpt_dest(const Waypoint* wpt_orig, const Waypoint* wpt_adj) const;
+  Waypoint* create_wpt_dest(const Waypoint* wpt_orig, const Waypoint* wpt_orig_adj) const;
   int is_small_angle(const Waypoint* wpt_orig,
                      const Waypoint* wpt_orig_prev,
                      const Waypoint* wpt_orig_next) const;
diff --git a/garmin_tables.cc b/garmin_tables.cc
index 09bd94576..0560b96c3 100644
--- a/garmin_tables.cc
+++ b/garmin_tables.cc
@@ -730,9 +730,9 @@ gt_color_value(const unsigned int garmin_index)
 uint32_t
 gt_color_value_by_name(const QString& name)
 {
-  for (int i = 0; i < gt_colors.size(); ++i) {
-    if (QString::compare(gt_colors[i].name, name, Qt::CaseInsensitive) == 0) {
-      return gt_colors[i].rgb;
+  for (const auto& gt_color : gt_colors) {
+    if (QString::compare(gt_color.name, name, Qt::CaseInsensitive) == 0) {
+      return gt_color.rgb;
     }
   }
 
diff --git a/humminbird.h b/humminbird.h
index 9892e9b29..67a210345 100644
--- a/humminbird.h
+++ b/humminbird.h
@@ -165,15 +165,14 @@ private:
 
   void humminbird_rte_head(const route_head* rte);
   void humminbird_rte_tail(const route_head* rte);
-  static QString wpt_to_id(const Waypoint*);
+  static QString wpt_to_id(const Waypoint* wpt);
   void humminbird_write_rtept(const Waypoint* wpt) const;
   void humminbird_write_waypoint(const Waypoint* wpt);
   void humminbird_write_waypoint_wrapper(const Waypoint* wpt);
 
   /* Data Members */
 
-  QVector<arglist_t> humminbird_args = {
-  };
+  QVector<arglist_t> humminbird_args;
 };
 
 class HumminbirdHTFormat : public Format, private HumminbirdBase
@@ -214,8 +213,7 @@ private:
 
   /* Data Members */
 
-  QVector<arglist_t> humminbirdht_args = {
-  };
+  QVector<arglist_t> humminbirdht_args;
 };
 
 #endif // HUMMINBIRD_H_INCLUDED_
diff --git a/igc.h b/igc.h
index edda6616e..ff0ca964e 100644
--- a/igc.h
+++ b/igc.h
@@ -87,7 +87,7 @@ public:
   // Qt6 falls back to std::hash, but it may not use the seed.
   friend size_t qHash(const igc_ext_type_t& key, size_t seed = 0) noexcept
   {
-    return qHash(static_cast<std::underlying_type<igc_ext_type_t>::type>(key), seed);
+    return qHash(static_cast<std::underlying_type_t<igc_ext_type_t>>(key), seed);
   }
 
   QVector<arglist_t>* get_args() override
diff --git a/reverse_route.h b/reverse_route.h
index 4834c78b8..8ae15e8db 100644
--- a/reverse_route.h
+++ b/reverse_route.h
@@ -40,8 +40,7 @@ public:
 
 private:
   int prev_new_trkseg{};
-  QVector<arglist_t> args = {
-  };
+  QVector<arglist_t> args;
 
   void reverse_route_wpt(const Waypoint* waypointp);
   void reverse_route_head(const route_head* rte);
diff --git a/src/core/nvector.cc b/src/core/nvector.cc
index f6e8ad894..17cd4b962 100644
--- a/src/core/nvector.cc
+++ b/src/core/nvector.cc
@@ -31,10 +31,10 @@
 namespace gpsbabel
 {
 
-LatLon::LatLon(double latitude, double longitude)
+LatLon::LatLon(double latitude, double longitude) : lat(latitude), lon(longitude)
 {
-  lat = latitude;
-  lon = longitude;
+  
+  
 }
 
 NVector::NVector(double latitude_degrees, double longitude_degrees)
diff --git a/swapdata.h b/swapdata.h
index 0f829f8bb..ce5f9cf44 100644
--- a/swapdata.h
+++ b/swapdata.h
@@ -40,8 +40,7 @@ public:
   void process() override;
 
 private:
-  QVector<arglist_t> args = {
-  };
+  QVector<arglist_t> args;
 
   static void swapdata_cb(const Waypoint* ref);
 
diff --git a/xmlgeneric.cc b/xmlgeneric.cc
index 29c8bf362..8d4d3e755 100644
--- a/xmlgeneric.cc
+++ b/xmlgeneric.cc
@@ -67,7 +67,7 @@ XmlGenericReader::xml_tbl_lookup(const QString& tag, xg_cb_type cb_type)
 
 void
 XmlGenericReader::xml_common_init(const QString& fname, const char* encoding,
-         const QStringList ignorelist, const QStringList skiplist)
+         const QStringList& ignorelist, const QStringList& skiplist)
 {
   rd_fname = fname;
 
diff --git a/xmlgeneric.h b/xmlgeneric.h
index dd1bb86a5..c4e041338 100644
--- a/xmlgeneric.h
+++ b/xmlgeneric.h
@@ -89,8 +89,8 @@ public:
   template<class MyFormat>
   void xml_init(const QString& fname, MyFormat* instance, const QList<xg_fmt_map_entry<MyFormat>>& tbl,
                 const char* encoding = nullptr,
-                const QStringList ignorelist = QStringList(),
-                const QStringList skiplist = QStringList())
+                const QStringList& ignorelist = QStringList(),
+                const QStringList& skiplist = QStringList())
   {
     build_xg_tag_map(instance, tbl);
 
@@ -165,7 +165,7 @@ private:
 
   XgCallbackBase* xml_tbl_lookup(const QString& tag, xg_cb_type cb_type);
   void xml_common_init(const QString& fname, const char* encoding,
-                       const QStringList ignorelist, const QStringList skiplist);
+                       const QStringList& ignorelist, const QStringList& skiplist);
   xg_shortcut xml_shortcut(QStringView name);
   void xml_run_parser(QXmlStreamReader& reader);